-
Notifications
You must be signed in to change notification settings - Fork 231
[Remove Vuetify from Studio] 'Something went wrong' error page in Channels #5313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Remove Vuetify from Studio] 'Something went wrong' error page in Channels #5313
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything is looking good, except that the Back to Home button is not functional.
:text="$tr('refreshAction')" | ||
@click="reloadPage" | ||
/> | ||
<KButton |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To maintain the link-type behaviour here, this needs to be a KRouterLink instead of a KButton, with the backHomeLink
object using v-bind
instead:
```suggestion
<KRouterLink
v-bind="backHomeLink"
appearance="raised-button"
:text="$tr('backToHomeAction')"
/>
Alternatively, if that doesn't play nicely with KButtonGroup
you can bind the @click event of the KButton to trigger programmatic navigation with Vue Router https://v3.router.vuejs.org/guide/essentials/navigation.html
@click="$router.push(backHomeLink)"
in place of the v-bind.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right, ill update it
@Kartikayy007 are you still planning to push the requested change? |
cca59b0
to
1708d0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Final comment addressed.
Summary
Removed Vuetify dependencies from the
GenericError.vue
component by replacing them with Kolibri Design System (KDS) components.Changes Made
AppError
withStudioAppError
(Vuetify-free version).VLayout
wrapper withKButtonGroup
for semantic button grouping.VBtn
components with:KButton
(Refresh)KRouterLink
(Navigation)Manual Verification
ChannelListIndex.vue
to force display error page (v-if="true"
).Screenshots
After (with KDS):

References
Reviewer Guidance
How to Test